Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

$DEVICE naming changes #1356

Merged
merged 2 commits into from Mar 14, 2017
Merged

$DEVICE naming changes #1356

merged 2 commits into from Mar 14, 2017

Conversation

lrusak
Copy link
Member

@lrusak lrusak commented Feb 16, 2017

First we want to be able to build the base project by itself. The $DEVICE should be an overlay of the base project, not it's own.

The naming should use $DEVICE.$ARCH not $PROJECT.$DEVICE.$ARCH. This makes it so we don't have to change any backend or updated things.

ping @codesnake @kszaq @kivutar and whoever else may use the $DEVICE call that this may change the way their build works.

@kivutar
Copy link
Contributor

kivutar commented Feb 16, 2017

Your change looks OK to me.
We should also think about using $DEVICE over $SYSTEM, or the opposite, since they are similar mechanism. But that's another discussion.

@vpeter4
Copy link
Contributor

vpeter4 commented Feb 17, 2017

Like I wrote: SYSTEM has nothing to do with DEVICE. The only reason I introduce SYSTEM was to distinguish which final image to build because of different bootloader has to be used. All other pieces are the same. That's why it is not needed to build multiple projects just because of one (or few) different package.
But if this has to change than just do it.

@kivutar
Copy link
Contributor

kivutar commented Feb 17, 2017

OK I understand the difference now. Thanks.

@chewitt
Copy link
Member

chewitt commented Feb 28, 2017

@lrusak can you rebase this pls.

@lrusak
Copy link
Member Author

lrusak commented Feb 28, 2017

rebased.

@lrusak
Copy link
Member Author

lrusak commented Mar 14, 2017

bueller?

Copy link
Member

@chewitt chewitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was busy crashing the Ferrari..

@chewitt chewitt merged commit 5f2a479 into LibreELEC:master Mar 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants